Add Ramp logo to website (LFMKT-2245) - #3493
Conversation
Include Ramp in the homepage enterprise logo grid, adopters table, and wrapped customers section (LFMKT-2245). Co-authored-by: annabellscha <annabellscha@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Keep expedia.svg and comment out its import/usage so it can be restored later. Ramp takes Expedia's slot to preserve a full 6×2 grid. Co-authored-by: annabellscha <annabellscha@users.noreply.github.com>
dosubot
Bot
added
the
size:S
|
@claude review |
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
LGTM — straightforward customer logo swap.
Extended reasoning...
Overview
Swaps Expedia for Ramp across three UI surfaces (EnterpriseLogoGrid.tsx, Customers.tsx), adds a new ramp.svg wordmark asset matching the existing 140×40 gray-wordmark format, and adds one row to the public adopters markdown table. Expedia's import/usage is commented out (not deleted) for easy restore, which is a reasonable, low-risk approach for a marketing asset swap.
Security risks
None. This is a static asset and presentational data change — no user input, no auth, no data handling logic touched.
Level of scrutiny
Low. This is a content/marketing change (logo grid, wrapped customers list, adopters table) with no logic branches, no md-override counterpart to keep in sync, and the new SVG follows the same structure/dimensions as sibling logos in the same directory.
Other factors
Checked that the new "Ramp" entries are consistently threaded through all three consuming files (EnterpriseLogoGrid.tsx grid, Customers.tsx logo list + hardcoded order array) and that the adopters table row is alphabetically placed and formatted consistently with neighboring rows. No outstanding review comments in the timeline to address.
2 participants
Summary
Adds the Ramp logo to the Langfuse website now that logo usage permission has been confirmed (LFMKT-2245).
components/home/img/ramp.svgin the same 140×40 gray wordmark format as other enterprise logosEnterpriseLogoGrid(keeps a full 6×2 grid)expedia.svgin the repo and comment out its import/usage for easy restore laterTest plan
expedia.svgremains in the repo; Expedia usage is commented out rather than deletedReview notes — logo placement
Ramp sits in Expedia’s former slot (bottom row, between Cisco and Rocket Money). Visual check: size, centering, and gray contrast match neighboring logos; no clipping or oversized whitespace.
Homepage logo grid with Ramp replacing Expedia
Close-up: Cisco, Ramp, Rocket Money
To show artifacts inline, enable in settings.
Linear Issue: LFMKT-2245
Greptile Summary
Adds Ramp to the website’s customer references and logo displays after logo-use permission was confirmed.
Confidence Score: 5/5
The PR appears safe to merge with no actionable defects identified.
The new logo follows existing asset and rendering conventions, and every added Ramp lookup resolves to a correctly shaped record without changing surrounding behavior.
Reviews (1): Last reviewed commit: "Replace Expedia with Ramp in homepage lo..." | Re-trigger Greptile